DOM update MessageEventTarget missed this parameters#61086
DOM update MessageEventTarget missed this parameters#61086sandersn merged 2 commits intomicrosoft:mainfrom
Conversation
|
@typescript-bot test this |
|
@sandersn Here are the results of running the user tests with tsc comparing Everything looks good! |
|
Hey @sandersn, the results of running the DT tests are ready. Everything looks the same! |
|
@sandersn Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
@sandersn Here are the results of running the top 400 repos with tsc comparing Everything looks good! |
|
matching dom lib generator PR: microsoft/TypeScript-DOM-lib-generator#1894 @typescript-bot test this |
|
@sandersn Here are the results of running the user tests with tsc comparing Everything looks good! |
|
Hey @sandersn, the results of running the DT tests are ready. Everything looks the same! |
|
@sandersn Here they are:
tscComparison Report - baseline..pr
System info unknown
Hosts
Scenarios
Developer Information: |
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
@sandersn Here are the results of running the top 400 repos with tsc comparing Everything looks good! |
While trying to update typescript-dom-lib-generator so that it generates the correct types for MessageEventTarget's this parameters, I realised that my manual fix missed a couple. They're not required to make the known problem code in DT/top400 compile, but all the
thisparameters should have consistent types.Of course, the whole thing is so fragile that I want to run tests as I figure out how to make dom-lib-generator generate the correct types, so that I can know if I need an even-more-complex fix.